Skip to content

Submitter module#875

Merged
gthea merged 16 commits intodevelopmentfrom
submitter-module
Mar 23, 2026
Merged

Submitter module#875
gthea merged 16 commits intodevelopmentfrom
submitter-module

Conversation

@gthea
Copy link
Contributor

@gthea gthea commented Mar 17, 2026

Android SDK

What did you accomplish?

  • Extracted submitter module.
  • Consolidated the duplicated submitter code into RecorderTask. This includes all periodic submissions, such as events, impressions and unique keys (telemetry not included since it follows a different pattern).
  • RecorderTask pops from storage, submits and removes from storage on successful submission.
  • This module does not have networking, consumers need to implement a RecorderSubmitter to post.

@gthea gthea self-assigned this Mar 17, 2026
@devops-split
Copy link

devops-split commented Mar 17, 2026

Android Diffuse Analysis Results

Source Branch: submitter-module
Target Branch: development
Commit: 867cae1232778909d354f98f0fc595b4290437ef

Script Output:

 AAR      │ old     │ new     │ diff   
──────────┼─────────┼─────────┼────────
      jar │ 1.1 MiB │ 1.1 MiB │ +995 B 
 manifest │   213 B │   213 B │    0 B 
    other │ 2.9 KiB │ 2.9 KiB │    0 B 
──────────┼─────────┼─────────┼────────
    total │ 1.2 MiB │ 1.2 MiB │ +995 B 

 JAR     │ old  │ new  │ diff          
─────────┼──────┼──────┼───────────────
 classes │  933 │  940 │  +7 (+12 -5)  
 methods │ 5759 │ 5771 │ +12 (+57 -45) 
  fields │ 2642 │ 2639 │  -3 (+29 -32)

@devops-split
Copy link

devops-split commented Mar 17, 2026

Android Diffuse Analysis Results

Source Branch: submitter-module
Target Branch: development
Commit: 28881015be710a9b8cd86afb6287334086c0315c

Script Output:

 AAR      │ old     │ new     │ diff   
──────────┼─────────┼─────────┼────────
      jar │ 1.1 MiB │ 1.1 MiB │ +995 B 
 manifest │   213 B │   213 B │    0 B 
    other │ 2.9 KiB │ 2.9 KiB │    0 B 
──────────┼─────────┼─────────┼────────
    total │ 1.2 MiB │ 1.2 MiB │ +995 B 

 JAR     │ old  │ new  │ diff          
─────────┼──────┼──────┼───────────────
 classes │  933 │  940 │  +7 (+12 -5)  
 methods │ 5759 │ 5771 │ +12 (+57 -45) 
  fields │ 2642 │ 2639 │  -3 (+29 -32)

@sonarqube-pull-requests
Copy link

sonarqube-pull-requests bot commented Mar 17, 2026 •
edited
Loading

Quality Gate passed Quality Gate passed

Issues
0 New issues
10 Fixed issues
2 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@gthea gthea marked this pull request as ready for review March 17, 2026 22:48
@gthea gthea requested a review from a team as a code owner March 17, 2026 22:48
@gthea gthea merged commit 03413e6 into development Mar 23, 2026
5 of 7 checks passed
@gthea gthea deleted the submitter-module branch March 23, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ZamoraEmmanuel ZamoraEmmanuel ZamoraEmmanuel approved these changes

Assignees

@gthea gthea

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gthea @devops-split @ZamoraEmmanuel